Documentation Review Changes - #93
Conversation
… since all data was a duplicate.
|
@rogurr, this looks like the right direction. Please let me know when it's ready for review (since it is currently in draft). |
…tter references to the Patina documentation
@makubacki, I'm going to ping Phil and Jerry about this PR and let them decide to take it as is or use it in one of their PRs since they are responsible for the full documentation update. This is just a confirmation that the data in the files deleted did not contain any information that is not already in the Patina documents. I documented in each task why I could remove the file and linked as appropriate in the repo or the task. |
Sounds good. I'm mainly interested in looking through the Patina-related content being deleted and confirming that it's all gone. If that's already done, great. I'd expect anything to be really outdated and inconsequential to the Patina docs. |
philgweber
left a comment
philgweber
left a comment
There was a problem hiding this comment.
I agree in removing these since they are all documented in Patina already.
There was a problem hiding this comment.
Pull request overview
This PR removes Patina- and Secure Boot-related documentation pages from this repository (now hosted in the Patina repository/book) and updates navigation and in-book links to point to the external Patina Book and Patina source repository. It also updates local build/publish automation and ignore rules to stop building/copying the removed Patina track content.
Changes:
- Removed the Patina track book content and the Integrator Patina integration page from this repo.
- Removed duplicated Secure Boot documentation pages and references from summaries/navigation.
- Updated guide pages to link to the external Patina Book and Patina GitHub repository; updated build scripts and GitHub Actions workflow to stop building the Patina track.
Reviewed changes
Copilot reviewed 20 out of 22 changed files in this pull request and generated 1 comment.
Show a summary per file
| File | Description |
|---|---|
tracks/security/src/SUMMARY.md |
Removes the Secure Boot entry from the security track TOC. |
tracks/security/src/secure_boot.md |
Removes duplicated Secure Boot page from the security track. |
tracks/patina/src/track_overview.md |
Removes Patina track overview page (track being removed). |
tracks/patina/src/SUMMARY.md |
Removes Patina track SUMMARY (track being removed). |
tracks/patina/src/patina_concepts.md |
Removes duplicated Patina background content (now external). |
tracks/patina/src/more_patina.md |
Removes Patina resource-links page (now external). |
tracks/integrator/src/SUMMARY.md |
Removes Patina integration entry from integrator track TOC. |
tracks/integrator/src/patina.md |
Removes Patina integration page (now external). |
guide_book/src/tracks.md |
Updates Patina track link to the external Patina Book URL. |
guide_book/src/SUMMARY.md |
Removes Patina- and Secure Boot-related entries from the main guide TOC. |
guide_book/src/overview.md |
Replaces Patina Track reference with links to Patina Book + source repo. |
guide_book/src/laptop/integrating.md |
Updates Patina link in the “Add a Patina Boot Layer” step to external Patina Book. |
guide_book/src/intro/welcome.md |
Updates Patina pointers to external Patina Book + source repo. |
guide_book/src/intro/getting_started.md |
Updates Patina developer guidance row to external Patina Book + source repo. |
guide_book/src/intro/concepts/patina.md |
Removes duplicated Patina concepts page (now external). |
guide_book/src/architecture/secure_boot.md |
Removes duplicated Secure Boot architecture page (now external/elsewhere). |
guide_book/src/architecture/patina_framework.md |
Removes Patina framework architecture page (now external). |
guide_book/src/architecture/patina_components.md |
Removes Patina components page (now external). |
.temp/build.bat |
Stops copying/building the Patina track in the local build script. |
.gitignore |
Removes ignore entries for Patina track generated/duplicated files. |
.github/workflows/publish-guide-book.yml |
Removes Patina track from the track build/publish loop. |
💡 Add a code-review agent skill or configure MCP servers for context-aware, tailored reviews. Learn more in the docs.
Removed specific files no longer needed due to documentation in the Patina repository.
This PR is just removing those files and fixing up links to the proper Patina book pages, it does not address content within the existing files.